Skip to content

[WEDO-6992] Removing cookies from preview#2686

Merged
lukdog merged 1 commit intomainfrom
lukdog-remove-cookies
Oct 27, 2025
Merged

[WEDO-6992] Removing cookies from preview#2686
lukdog merged 1 commit intomainfrom
lukdog-remove-cookies

Conversation

@lukdog
Copy link
Copy Markdown
Member

@lukdog lukdog commented Oct 24, 2025

What This PR Changes

cookie banner in previews is annoying, I'm trying to remove it

Contribution Guidelines

@lukdog lukdog added the preview label Oct 24, 2025
@github-actions
Copy link
Copy Markdown

Preview Deployment

🚀 Preview this PR: https://68fb92374e041c009a1d9d70--docs-content.netlify.app
📍 Commit SHA: f648e9a

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the cookie banner from preview deployments by disabling the Hotjar CDN during the build process. The change sets an empty value for the GATSBY_HF_CDN_URL environment variable when building previews.

Key Changes

  • Modified the build command in the preview workflow to set GATSBY_HF_CDN_URL="", which prevents the cookie banner from appearing in preview deployments

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@jhansson-ard jhansson-ard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful 🥹

@lukdog lukdog merged commit 2bff837 into main Oct 27, 2025
7 checks passed
@lukdog lukdog deleted the lukdog-remove-cookies branch October 27, 2025 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants